Skip to content

fix: improve setup wizard and docs - #21

Merged
sweden-snow merged 2 commits into
mainfrom
fix/setup-wizard
Apr 10, 2026
Merged

fix: improve setup wizard and docs#21
sweden-snow merged 2 commits into
mainfrom
fix/setup-wizard

Conversation

@sweden-snow

Copy link
Copy Markdown
Contributor
  • Add Claude Code CLI detection: --global uses claude mcp add when available
  • Add PII mode and dry-run prompts to the wizard
  • Create config directories if they don't exist (fixes ENOENT on fresh installs)
  • Restructure Getting Started docs: wizard first, manual config second
  • Clarify --local flag is for maintainers only
  • Update keychain docs to reflect claude mcp add support

 - Add Claude Code CLI detection: --global uses `claude mcp add` when available
 - Add PII mode and dry-run prompts to the wizard
 - Create config directories if they don't exist (fixes ENOENT on fresh installs)
 - Restructure Getting Started docs: wizard first, manual config second
 - Clarify --local flag is for maintainers only
 - Update keychain docs to reflect `claude mcp add` support

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Improves the setup experience for configuring the Crystallize MCP server across Claude Code and Claude Desktop, and updates documentation to prioritize the interactive wizard and clarify maintainer-only flows.

Changes:

  • Extends the setup wizard with PII mode + dry-run prompts and ensures config directories are created before writing.
  • Adds Claude Code CLI detection for --global to prefer claude mcp add when available, with Desktop config fallback when not.
  • Restructures README Getting Started (wizard-first) and updates keychain + maintainer (--local) guidance.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
tests/schema.test.ts Renames unused callback params to avoid linter no-shadow warnings.
src/bin/setup.ts Adds PII/dry-run prompts, directory creation before writes, and Claude CLI-based global registration.
README.md Reorders Getting Started to emphasize the wizard and updates Claude/keychain/maintainer documentation.
.oxlintrc.json Enables no-shadow (warn) to catch shadowed identifiers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@sweden-snow
sweden-snow merged commit ece5c05 into main Apr 10, 2026
1 check passed
@sweden-snow
sweden-snow deleted the fix/setup-wizard branch April 10, 2026 12:50
istonejoeljonsson added a commit that referenced this pull request Apr 13, 2026
* 'main' of github.com:HayoDev/crystallize-mcp:
  chore(main): release crystallize-mcp 0.2.0 (#24)
  feat!: setup-wizard :nail-care: (#23)
  chore(main): release crystallize-mcp 0.1.10 (#22)
  fix: improve setup wizard and docs (#21)
  chore(main): release crystallize-mcp 0.1.9 (#20)
  feat: support content chunk updates with dot notation and fix richTex… (#18)
istonejoeljonsson added a commit that referenced this pull request Apr 13, 2026
* main:
  chore(main): release crystallize-mcp 0.2.0 (#24)
  feat!: setup-wizard :nail-care: (#23)
  chore(main): release crystallize-mcp 0.1.10 (#22)
  fix: improve setup wizard and docs (#21)
  chore(main): release crystallize-mcp 0.1.9 (#20)
  feat: support content chunk updates with dot notation and fix richTex… (#18)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants